fix: correctly generate test x request id for each test in one browser #825
Conversation
DudaGod
force-pushed
the
HERMIONE-1328.fix_x_req_id
branch
from
7572601 to
443056b
Compare
|
|
||
| this._config = config.forBrowser(this.id); | ||
| this._debug = config.system.debug; | ||
| this._session = null; | ||
| this._callstackHistory = null; | ||
| this._state = { | ||
| ...opts.state, |
There was a problem hiding this comment.
Поле теперь прокидываю в state. Посчитал, что нет необходимости в отдельном поле.
| this._browser = await this._browserAgent.getBrowser({ state }); | ||
|
|
||
| // use correct state for cached browsers | ||
| if (this._browser.state.testXReqId !== state.testXReqId) { |
There was a problem hiding this comment.
Бага исправляется за счет этого куска кода. Сюда из браузер пулов может прилететь закешированный браузер в котором находится свой testXReqId и мне его нужно переопределить.
Мне этот if не очень нравится, но нормального решения не нашел. Пытался прокидывать state через массу браузер пулов (cached, limited, basic, ...), но выглядит очень топорно и сложно. Поэтому решил остановится на этом варианте.
There was a problem hiding this comment.
а как эти testXReqId-ы вообще могут совпасть, если ты в state-е его новый генеришь?
There was a problem hiding this comment.
Они не совпадают. Проблема возникает когда используется 1 инстанс браузера на N тестов. В первом тесте выполняется генерация testXReqId и запись его в состояние браузера. А следующий тест уже использует существующий инстанс браузера в котором имеется testXReqId от первого теста. В итоге testXReqId от второго теста никуда не попадает. Т.е. он генерится, но из cached пула возвращается существующий браузер.
Как вариант можно выполнять applyState внутри этого самого caching пула.
| @@ -69,7 +69,6 @@ module.exports = class ExistingBrowser extends Browser { | |||
|
|
|||
| quit() { | |||
| this._meta = this._initMeta(); | |||
| this._state = { isBroken: false }; | |||
There was a problem hiding this comment.
Вот это бесполезная строка была. Решил удалить ее.
In previous PR - #819 I did not take into account the case with the correct specification of
testXReqIdwhen using the same browser for several tests. The problem was that the browser was cached and its fieldtestXReqIdwas not modified for next test.